Skip to content

Conversation

@forsyth2
Copy link
Collaborator

Summary

Objectives:

  • Streamline the blocking/non-blocking test.

Issue resolution:

Select one: This pull request is...

  • a bug fix: increment the patch version
  • a small improvement: increment the minor version
  • a new feature: increment the minor version
  • an incompatible (non-backwards compatible) API change: increment the major version

Small Change

  • To merge, I will use "Squash and merge". That is, this change should be a single commit.
  • Logic: I have visually inspected the entire pull request myself.
  • Pre-commit checks: All the pre-commits checks have passed.

@forsyth2 forsyth2 self-assigned this Oct 20, 2025
@forsyth2 forsyth2 added the Testing Files in `tests` modified label Oct 20, 2025
@forsyth2 forsyth2 changed the title Issue 385 blocking test Improvements to blocking test Oct 20, 2025
@forsyth2 forsyth2 force-pushed the issue-385-blocking-test branch from 9408a54 to 96b3f8f Compare October 20, 2025 21:59
@forsyth2 forsyth2 force-pushed the issue-385-blocking-test branch from 96b3f8f to fca1525 Compare October 20, 2025 22:16
@forsyth2
Copy link
Collaborator Author

Still in progress. Remaining action items:

  • Add assertions that certain lines appear/don't appear in the logs
  • Make sure https://app.globus.org/activity shows successful transfers
  • Make sure the BLOCKING and NON-BLOCKING cases work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Testing Files in `tests` modified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants